feat: generate a k8s_name for test#87
Conversation
|
Another thing I found when working on this PR: There's a I think update |
Signed-off-by: Bryan <[email protected]>
Signed-off-by: Gabriele Bartolini <[email protected]>
|
Hi @ImSingee , I think the preferred way here would be to go ahead with #9383 and lift this limitation on the extension |
|
@NiccoloFei Yes, fixing it directly in cnpg will be better. But I think we can merge this PR first as that issue is just planned to resolve several months later. This is a test-only thing, we can easily revert this when the name with hypen is accpeted by cnpg. Also, merge this first can reduce unnecessary friction for other contributers. |
|
I'm abandoning this given that cloudnative-pg/cloudnative-pg#9386 has been merged and will be available in |
Some extensions' name contain
_, such aspg_ivm,pg_textsearch. Tests will fail on these extensions because this name violates k8s name's rule.This PR fixes this by generating a
k8s_namefield invalues.yamland use this name as all k8s resource names.